Support for recursive extends (adjusted)#24
Support for recursive extends (adjusted)#24HoldYourWaffle wants to merge 4 commits intoreshape:masterfrom
Conversation
|
When I tried to use this in my own project it didn't work as I expected (throwing an error that definitely shouldn't be there), currently investigating. |
|
After a lot of staring I think I found something that could lead to the cause. Keep in mind that I have no idea how reshape works beyond the code of this plugin, so there may be some (big) inaccuracies. I also don't have the code in front of me right now, so the variable names might be slightly inaccurate. It seems like the tree modification logic (changing just the contents of nodes) is not sufficient to support this behavior. The merged tree is the same before and after the merge (this is something I didn't understand at all), so the already merged blocks are still passed on to the 'next layer'. I'm way too tired to be doing this right now but I'll be looking at this again tomorrow. |
|
Let me know if you want to pair and walk through this whenever you're taking another look! |
|
Also happy to pair! |
|
@josephfinlayson shoot me an email escalantejeff at gmail and let's set up a time 😁 |
|
I'm going on vacation soon so I won't have any time to look at this until somewhere next week. |
(supersedes #22)
As mentioned in #22, I need this feature in my project. Sadly @josephfinlayson never made the adjustments you wanted, so I did it.
Changes relative to #22:
package.json)There are 3 things I'm not 100% sure I did correctly:
extends inherited layouttest is almost the exactly the same.